Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add support for multiple plone sites on one installation via PLONE_BACKEND_HOST #110

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add support for multiple plone sites on one installation via PLONE_BACKEND_HOST #110

wants to merge 5 commits into from

Conversation

maethu
Copy link
Member

@maethu maethu commented Apr 17, 2023 •
edited
Loading

In order to support multiple Plone sites on one zope instance. This PR introduces the PLONE_BACKEND_HOST env variable.

Details:

  • If PLONE_BACKEND_HOST is set, the PLONE_BACKEND env variable is ignored.
  • The PLONE_BACKEND_HOST needs to point to the root of the zope instance.
  • The URL to the Plone site is computed in the ES manager (Where we have access to the plone context)

@maethu maethu marked this pull request as ready for review April 17, 2023 18:09
@maethu maethu requested a review from ericof May 24, 2023 15:11
@maethu
Copy link
Member Author

maethu commented Sep 29, 2023

@ericof Do you mind having a look here as well?

@ericof
Copy link
Member

ericof commented Sep 27, 2024

@maethu I'm finally going to test it here

@maethu
Copy link
Member Author

maethu commented Nov 13, 2024

@maethu I'm finally going to test it here

Awesome, thanks! This branch is online for approx. 1 year, and we just lunched a new website www.baselland.ch with over 1 Mio requests very day. So the code is battle tested as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericof ericof Awaiting requested review from ericof

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@maethu @ericof